Skip to content

Use taiki-e/install-action and binstall in CI#17573

Merged
blaginin merged 2 commits intoapache:mainfrom
AdamGS:adamg/use-taiki-e-install-action-ci
Sep 15, 2025
Merged

Use taiki-e/install-action and binstall in CI#17573
blaginin merged 2 commits intoapache:mainfrom
AdamGS:adamg/use-taiki-e-install-action-ci

Conversation

@AdamGS
Copy link
Copy Markdown
Contributor

@AdamGS AdamGS commented Sep 15, 2025

Which issue does this PR close?

Rationale for this change

Should make CI faster. For example currently just installing cargo-msrv takes >5 minutes, with this PR its ~4 seconds.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the development-process Related to development process of DataFusion label Sep 15, 2025
@AdamGS AdamGS force-pushed the adamg/use-taiki-e-install-action-ci branch from 9a92103 to 5a63a6c Compare September 15, 2025 12:27
@AdamGS AdamGS marked this pull request as ready for review September 15, 2025 12:29
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
- name: Install cargo-audit
run: cargo install cargo-audit
uses: taiki-e/install-action@v2
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we put the version hash?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, give me a second

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@AdamGS AdamGS requested a review from blaginin September 15, 2025 12:51
Copy link
Copy Markdown
Member

@blaginin blaginin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@blaginin
Copy link
Copy Markdown
Member

blaginin commented Sep 15, 2025

I think in the future we could also use nextest to run tests!

One thing before merging, @Omega359, you wrote:

taiki-e/install-action@nextest is not an allowed action

I've read asf guide and I don't see it's being not allowed - is there something I'm missing?

@Omega359
Copy link
Copy Markdown
Contributor

It failed when I had it enabled. I somewhat doubt it would run well anyways given the machine constraints

@blaginin
Copy link
Copy Markdown
Member

blaginin commented Sep 15, 2025

Got it! I think it's working now (and actually uses less resources as tools are not built), so going to merge if that's ok with everyone. I think we can iterate on top (as in adding nextest and some more tools)?

@blaginin blaginin merged commit 7d08c27 into apache:main Sep 15, 2025
28 checks passed
@AdamGS AdamGS deleted the adamg/use-taiki-e-install-action-ci branch September 15, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use taiki-e/install-action in CI

3 participants